Skip to content

ENH: Make HDFStore iterable #12253

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

toobaz
Copy link
Member

@toobaz toobaz commented Feb 7, 2016

closes #12221

HDFStore is not an iterator - but being iterable, it can return an iterator of itself (i.e. of .keys()).

self.assertTrue(set(
store.keys()) == set(['/a', '/b', '/c', '/d', '/foo/bar']))
expected = set(['/a', '/b', '/c', '/d', '/foo/bar'])
self.assertTrue(set(store.keys()) == expected)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls add the tests from the original issue exactly (as a new test)

@jreback jreback added IO HDF5 read_hdf, HDFStore Bug API Design labels Feb 8, 2016
@jreback
Copy link
Contributor

jreback commented Feb 8, 2016

need a whatsnew

@toobaz toobaz force-pushed the iterable_hdfstore branch from 5701d21 to 23bfb3a Compare February 9, 2016 09:54
@jreback jreback added this to the 0.18.0 milestone Feb 11, 2016
@jreback jreback closed this in c805c3b Feb 11, 2016
@jreback
Copy link
Contributor

jreback commented Feb 11, 2016

@toobaz thanks!

@toobaz toobaz deleted the iterable_hdfstore branch February 11, 2016 06:21
cldy pushed a commit to cldy/pandas that referenced this pull request Feb 11, 2016
closes pandas-dev#12221    HDFStore is not an iterator - but being iterable, it
can return an iterator of itself (i.e. of ``.keys()``).

Author: Pietro Battiston <[email protected]>

Closes pandas-dev#12253 from toobaz/iterable_hdfstore and squashes the following commits:

23bfb3a [Pietro Battiston] ENH: Make HDFStore iterable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

KeyError Iterating on empty HDFStore
2 participants